Skip to content

Create keeper-dependent-features.md#165

Open
filimonov wants to merge 1 commit intomainfrom
filimonov-patch-2
Open

Create keeper-dependent-features.md#165
filimonov wants to merge 1 commit intomainfrom
filimonov-patch-2

Conversation

@filimonov
Copy link
Member

I have read the CLA Document and I hereby sign the CLA


## ON CLUSTER relation (important)

`ON CLUSTER` itself uses Keeper-backed distributed DDL queue (row 4).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
`ON CLUSTER` itself uses Keeper-backed distributed DDL queue (row 4).
`ON CLUSTER` itself relies on a Keeper-backed distributed DDL queue (see row 4).

## ON CLUSTER relation (important)

`ON CLUSTER` itself uses Keeper-backed distributed DDL queue (row 4).
Some features above already replicate through Keeper, so `ON CLUSTER` can be redundant:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Some features above already replicate through Keeper, so `ON CLUSTER` can be redundant:
Some of the features listed above already replicate via Keeper, which can make `ON CLUSTER` redundant:

| 15 | Experimental transactions | Configure `transaction_log.zookeeper_path` (and enable related experimental transaction settings). | Default: `/clickhouse/txn`. | `<path>/tail_ptr` and `<path>/log/csn-*` sequential nodes storing commit sequence and transaction IDs. |
| 16 | `Shared` database engine (Cloud) | ClickHouse Cloud managed behavior (not typically user-configured in self-managed OSS). | Internal/cloud-managed. | Shared catalog is Keeper-backed; low-level path layout is internal and not documented as a stable public contract. |

## ON CLUSTER relation (important)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## ON CLUSTER relation (important)
## How ON CLUSTER fits in (important)


## Notes

- For many rows, path names can be redirected to auxiliary Keeper clusters using `<auxiliary_zookeepers>` and `cluster_name:/path` notation where supported.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- For many rows, path names can be redirected to auxiliary Keeper clusters using `<auxiliary_zookeepers>` and `cluster_name:/path` notation where supported.
- For many features, path names can be redirected to auxiliary Keeper clusters using `<auxiliary_zookeepers>` and the `cluster_name:/path` notation, where supported.

## Notes

- For many rows, path names can be redirected to auxiliary Keeper clusters using `<auxiliary_zookeepers>` and `cluster_name:/path` notation where supported.
- Node names shown above are the stable conceptual layout from current source tree; some minor subnodes are version-specific.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Node names shown above are the stable conceptual layout from current source tree; some minor subnodes are version-specific.
- The metadata structure shown above reflects the stable conceptual layout from the current source tree; some minor subnodes may vary by version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants